Skip to content

Specify the retention length for artifacts#14395

Merged
brettcannon merged 1 commit intomicrosoft:mainfrom
brettcannon:upload-retention
Oct 14, 2020
Merged

Specify the retention length for artifacts#14395
brettcannon merged 1 commit intomicrosoft:mainfrom
brettcannon:upload-retention

Conversation

@brettcannon
Copy link
Copy Markdown
Member

@brettcannon brettcannon commented Oct 14, 2020

As discussed with Kim and Karthik:

  • 14 days for release branches (to cover time from cutting the branch to doing a release)
  • 7 days for main (to cover time between commits and potential consecutive failures preventing uploading to Azure Blob Storage)
  • 1 day for code coverage (as the workflow will upload to Codecov)

Helps keep resource usage down.
@brettcannon brettcannon added the no-changelog No news entry required label Oct 14, 2020
@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@codecov-io
Copy link
Copy Markdown

codecov-io commented Oct 14, 2020

Codecov Report

Merging #14395 into main will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #14395      +/-   ##
==========================================
- Coverage   59.40%   59.40%   -0.01%     
==========================================
  Files         716      716              
  Lines       39999    39999              
  Branches     5793     5793              
==========================================
- Hits        23763    23762       -1     
- Misses      14975    14976       +1     
  Partials     1261     1261              
Impacted Files Coverage Δ
src/client/common/utils/platform.ts 68.00% <0.00%> (-4.00%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1370b6d...ba6a1fd. Read the comment docs.

@brettcannon brettcannon merged commit 7b1281e into microsoft:main Oct 14, 2020
@brettcannon brettcannon deleted the upload-retention branch October 14, 2020 17:17
luabud pushed a commit to luabud/vscode-python that referenced this pull request Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants